[WIP] Remove needs: build from CI jobs #3154
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Plan: Remove "needs: build" from ci.yml jobs
Analysis
The
.github/workflows/ci.ymlfile currently has two jobs with unnecessaryneeds: builddependencies:testjob hasneeds: buildjsjob hasneeds: buildThese dependencies force the jobs to wait for the
buildjob to complete, but analysis shows:testuploads a coverage report)Changes Required
needs: buildfrom thetestjob (line 11)needs: buildfrom thejsjob (line 70)Expected Impact
Changes Made
✅ Removed
needs: buildfrom thetestjob✅ Removed
needs: buildfrom thejsjob✅ Validated YAML syntax with actionlint - passed
✅ Ran unit tests - all passed
✅ Verified build still works correctly
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.